MILABL-6766 Fix rapids env issues for non docker envs - #104
Merged
Conversation
blackcat
approved these changes
Aug 13, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Greptile Summary
The PR repairs RAPIDS environment creation on native backends by avoiding expression-renderer interpretation of
${PATH}, and explicitly packages a RAPIDS-compatible PyArrow release for targets where CUDA packages are skipped.$PATHso native environment rendering does not interpret{PATH}as an undefined expression.envVars— environment-variable declarations shared by Docker and native runenv consumers; the PATH entry changes from${PATH}to$PATH.Confidence Score: 5/5
The PR appears safe to merge, with no concrete changed-code defect established.
The PATH syntax avoids the documented native expression-renderer failure, and the added PyArrow version satisfies the documented RAPIDS constraint while targeting all configured platforms.
Important Files Changed
$PATHto prevent native expression-rendering failures while retaining Docker expansion.pyarrow==19.0.1dependency across the runenv’s configured targets.Reviews (1): Last reviewed commit: "MILABL-6766 Fix rapids env issues for no..." | Re-trigger Greptile
Context used: